Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replicate pull request 44 - Nicer Notebooks (#48) #49

Merged
merged 1 commit into from
Oct 18, 2023

Conversation

facusapienza21
Copy link
Member

  • Test suite with GitHub action and test coverage (Test suite with GitHub action and test coverage #46)

  • simple test suite

  • Create test.yml

  • Add passing test badge

  • add codedev action

  • Add directory to codecov action

  • add coverage option to pytest

  • install pytest-cov

  • Add covering badge

  • facusapienza21 -> PolarWandering

  • Trigger tests

  • Update codedev badge

  • run tests in both push and PR

  • Added MIT license badge

  • Sampling comparison notebook overhaul (Sampling comparison notebook overhaul #45)

  • progress on enhancing the explanations and ease of use of the sampling_comparison notebook

  • Add *.egg-info to .gitignore file to prevent them from being tracked

  • add Figure 3a to the intro cell and continue improvements

  • finish draft of user friendly sample comparison notebook

  • simplify egg-info ignored files

  • configure iPython kernel setup

  • test for sampling

  • test for estimation

  • update path of saved data

  • test theretical bound

  • Comments in init.py


* Test suite with GitHub action and test coverage (#46)

* simple test suite

* Create test.yml

* Add passing test badge

* add codedev action

* Add directory to codecov action

* add coverage option to pytest

* install pytest-cov

* Add covering badge

* facusapienza21 -> PolarWandering

* Trigger tests

* Update codedev badge

* run tests in both push and PR

* Added MIT license badge

* Sampling comparison notebook overhaul (#45)

* progress on enhancing the explanations and ease of use of the sampling_comparison notebook

* Add *.egg-info to .gitignore file to prevent them from being tracked

* add Figure 3a to the intro cell and continue improvements

* finish draft of user friendly sample comparison notebook

* simplify `egg-info` ignored files

* configure iPython kernel setup

* test for sampling

* test for estimation

* update path of saved data

* test theretical bound

* Comments in __init__.py

---------

Co-authored-by: Facundo Sapienza <fsapienza@berkeley.edu>
@codecov
Copy link

codecov bot commented Oct 18, 2023

Codecov Report

All modified lines are covered by tests ✅

❗ No coverage uploaded for pull request base (main@95ce1a4). Click here to learn what that means.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #49   +/-   ##
=======================================
  Coverage        ?   87.04%           
=======================================
  Files           ?        9           
  Lines           ?      301           
  Branches        ?        0           
=======================================
  Hits            ?      262           
  Misses          ?       39           
  Partials        ?        0           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@facusapienza21 facusapienza21 merged commit 57e98e6 into main Oct 18, 2023
3 checks passed
@facusapienza21 facusapienza21 deleted the docstring_updates branch October 18, 2023 18:55
facusapienza21 added a commit that referenced this pull request Dec 5, 2023
* update image path to be url rather than local path

* Update version of cartopy for JBook

* update logo to be relevant to this study

* Create intro markdown

The goal is to make an intro markdown file that quickly directs the
user to the most relevant notebooks.

* create installation.md

* simplify notebook explanations in grid

* Update toc given new files

Add the new intro and installation markdown files.
Adjust order of analysis notebooks.

---------

Co-authored-by: Nick Swanson-Hysell <swanson-hysell@berkeley.edu>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants